resharper fixes (#368)
authortsteven4 <13596209+tsteven4@users.noreply.github.com>
Tue, 18 Jun 2019 14:33:01 +0000 (08:33 -0600)
committerGitHub <noreply@github.com>
Tue, 18 Jun 2019 14:33:01 +0000 (08:33 -0600)
* fix "redundant statement", "redundant declaration"

* fix newline at eof.

28 files changed:
alan.cc
easygps.cc
garmin_gpi.cc
garmin_xt.cc
gpx.cc
gui/advdlg.h
gui/babeldata.h
gui/dpencode.cc
gui/filterdata.h
gui/filterwidgets.h
gui/format.h
gui/formatload.h
gui/gpx.cc
gui/gpx.h
gui/mainwindow.cc
gui/map.h
gui/processwait.cc
gui/processwait.h
gui/setting.h
jeeps/gpsapp.cc
jeeps/gpsserial.cc
jeeps/gpsusbcommon.cc
pcx.cc
route.cc
src/core/file.h
src/core/textstream.cc
src/core/textstream.h
xmlgeneric.cc

diff --git a/alan.cc b/alan.cc
index 020c48fd2ed6738918b5aa1f6e62f7b7881174c4..0a88e60075424f8650d10347af1d6f199087d9b6 100644 (file)
--- a/alan.cc
+++ b/alan.cc
@@ -490,11 +490,11 @@ static Waypoint* get_wpt(struct wprdata* wprdata, unsigned n)
   WP->latitude  = -pt2deg(wpt->pt.y);
   WP->longitude =  pt2deg(wpt->pt.x);
   WP->SetCreationTime(unpack_time(wpt->date, wpt->time));
-  for (j=WPT_NAME_LEN-1; j >= 0 && wpt->name[j] == ' '; j--) {};
+  for (j=WPT_NAME_LEN-1; j >= 0 && wpt->name[j] == ' '; j--) {}
   char *s = xstrndup(wpt->name,j+1);
   WP->shortname = s;
   xfree(s);
-  for (j=WPT_COMMENT_LEN-1; j >= 0 && wpt->comment[j] == ' '; j--) {};
+  for (j=WPT_COMMENT_LEN-1; j >= 0 && wpt->comment[j] == ' '; j--) {}
   if (j >= 0) {
     char *s = xstrndup(wpt->comment, j+1);
     WP->description = s;
@@ -541,11 +541,11 @@ static void wpr_read()
 
     route_head* RT = route_head_alloc();
     RT->rte_num = i;
-    for (j=RTE_NAME_LEN-1; j >= 0 && rte->name[j] == ' '; j--) {};
+    for (j=RTE_NAME_LEN-1; j >= 0 && rte->name[j] == ' '; j--) {}
     char *s = xstrndup(rte->name,j+1);
     RT->rte_name = s;
     xfree(s);
-    for (j=RTE_COMMENT_LEN-1; j >= 0 && rte->comment[j] == ' '; j--) {};
+    for (j=RTE_COMMENT_LEN-1; j >= 0 && rte->comment[j] == ' '; j--) {}
     if (j >= 0) {
       char *s = xstrndup(rte->comment,j+1);
       RT->rte_desc = s;
@@ -595,14 +595,14 @@ static void trl_read()
     route_head* TL = route_head_alloc();
     for (j=TRK_NAME_LEN-1;
          j >= 0 && (trkhdr->name[j] == ' ' || trkhdr->name[j] == '\0');
-         j--) {};
+         j--) {}
     char *s1 = xstrndup(trkhdr->name,j+1);
     TL->rte_name = s1;
     xfree(s1);
     /*  TL->rte_name[TRK_NAME_LEN+1] = 0; */   /* MAYBE BAD ADDRESS (Valgrind) */
     for (j=TRK_COMMENT_LEN-1;
          j >= 0 && (trkhdr->comment[j] == ' ' || trkhdr->comment[j] == '\0');
-         j--) {};
+         j--) {}
     s1 = xstrndup(trkhdr->comment,j+1);
     TL->rte_desc = s1;
     xfree(s1);
@@ -791,7 +791,7 @@ static void trl_track_hdr(const route_head* TL)
 
   struct trkhdr* trkhdr = TRL.loghdr.trkhdr;
 
-  for (idx=0; idx< MAXTRK && trkhdr[idx].occupied != TRK_UNUSED; idx++) {};
+  for (idx=0; idx< MAXTRK && trkhdr[idx].occupied != TRK_UNUSED; idx++) {}
   if (idx >= MAXTRK) {
     fatal(MYNAME ": Can't store more than %u tracklogs", MAXTRK);
   }
index 52bf2b5526efbed1ddee8609d91fe30224c428aa..c6ff725c54b619406c63dc944eb53c41a08fab19 100644 (file)
@@ -136,7 +136,6 @@ data_read()
         break;
       default:
         printf("Unknown tag %x\n", tag);
-        ;
       }
     }
     if (!link.url_.isEmpty() || !link.url_link_text_.isEmpty()) {
index 0452b198f1d01b3d64f2f36b25a77c8c1dd4ba11..bddea7fba91ba37366934581f002f7c041e6b00e 100644 (file)
@@ -1583,7 +1583,7 @@ garmin_gpi_read()
     if (! read_tag("garmin_gpi_read", tag, nullptr)) {
       return;
     }
-  };
+  }
 }
 
 
index 7406cc9f233912397b22aff954fd025f031ad40d..4d3ac4caa18d766e699c03ecf81ce18b9cdacce1 100644 (file)
@@ -365,11 +365,11 @@ format_garmin_xt_proc_atrk()
     double LatF = Lat;
     if (LatF>8388608) {
       LatF -= 16777216;
-    };
+    }
     double LonF = Lon;
     if (LonF>8388608) {
       LonF -= 16777216;
-    };
+    }
     double AltF = (double)uu * GARMIN_XT_ELE - 1500;
 
     //create new waypoint
diff --git a/gpx.cc b/gpx.cc
index 88c3009dbb4201235dc87ab50fecd34ccdaf6953..5f126a840fdd4ab4b2d2b48707e9852740786ebc 100644 (file)
--- a/gpx.cc
+++ b/gpx.cc
@@ -247,12 +247,12 @@ gpx_add_to_global(QStringList& ge, const QString& s)
 static inline QString toString(double d)
 {
   return QString::number(d, 'f', 9);
-};
+}
 
 static inline QString toString(float f)
 {
   return QString::number(f, 'f', 6);
-};
+}
 
 
 /*
index 10b6c24eb6907a80f937cb0bf5a9a43592c132ba..bf4fe7a1bc49ce9d378cb9b2429ee4f0e7d63c05 100644 (file)
@@ -36,7 +36,7 @@ public:
   QPushButton* formatButton()
   {
     return ui_.formatButton;
-  };
+  }
 
 private:
   Ui_AdvUi ui_;
index 9f58cc626c346ef8d8bac4ef5bf46713a30b9406..198f88893ed5a72858b76f7412d2be317b619911 100644 (file)
@@ -71,7 +71,7 @@ public:
     disableDonateDialog_(false),
     donateSplashed_(QDateTime(QDate(2010, 1, 1), QTime(0, 0, 0)))
   {
-  };
+  }
 
   void saveSettings(QSettings& st)
   {
index 882cc70a751976fd51f360fab7a3f337034006ae..fc07855ae27fcc1c6506e92c35a1fd8f7988909d 100644 (file)
@@ -100,7 +100,7 @@ void PolylineEncoder::createEncodings(string& encoded_points,
                                       const vector <LatLng>& points,
                                       const vector <double>& dists)
 {
-  encoded_points = "";;
+  encoded_points = "";
   int plat = 0;
   int plng = 0;
   for (unsigned int i = 0; i < points.size(); i++) {
index 5132493063797ce9ec1a7d30dfa2a0a3ae7eb03b..21310c8f98f2ffba25cbde1f2ef4a09edfb4bf13 100644 (file)
@@ -33,8 +33,8 @@
 class FilterData
 {
 public:
-  FilterData(): inUse_(true) {};
-  virtual ~FilterData() {};
+  FilterData(): inUse_(true) {}
+  virtual ~FilterData() {}
 
   void saveSettings(QSettings& st)
   {
index 10a1c0ab117b6a6f3720ee714401fe3832c48195..15ee0f8f7a95369cd29212e26fec89a512187699 100644 (file)
@@ -93,8 +93,8 @@ private:
 class FilterOption
 {
 public:
-  FilterOption() {};
-  virtual ~FilterOption() {};
+  FilterOption() {}
+  virtual ~FilterOption() {}
   virtual void setWidgetValue() = 0;
   virtual void getWidgetValue() = 0;
 };
index 708bcbda4e6618a73130dedeaea8c6c9bcd8898d..9954d492d88b0ed7b29fa46d48e86d596268ac03 100644 (file)
@@ -110,15 +110,17 @@ public:
   void setValue(QVariant v)
   {
     value_ = v;
-  };
+  }
+
   void setSelected(bool v)
   {
     isSelected_ = v;
-  };
+  }
+
   QString getHtml() const
   {
     return html_;
-  };
+  }
 
 private:
   QString name_;
@@ -155,7 +157,7 @@ public:
   {
     inputOptions_.clear();
     outputOptions_.clear();
-  };
+  }
 
   Format(const QString& name,
          const QString& description,
@@ -196,93 +198,107 @@ public:
   {
   }
 
-  ~Format() {};
+  ~Format() {}
 
   bool isReadWaypoints() const
   {
     return readWaypoints_;
-  };
+  }
+
   bool isReadTracks() const
   {
     return readTracks_;
-  };
+  }
+
   bool isReadRoutes() const
   {
     return readRoutes_;
-  };
+  }
+
   bool isReadSomething() const
   {
     return isReadWaypoints() || isReadTracks() || isReadRoutes();
-  };
+  }
 
   bool isWriteWaypoints() const
   {
     return writeWaypoints_;
-  };
+  }
+
   bool isWriteTracks() const
   {
     return writeTracks_;
-  };
+  }
+
   bool isWriteRoutes() const
   {
     return writeRoutes_;
-  };
+  }
+
   bool isWriteSomething() const
   {
     return isWriteWaypoints() || isWriteTracks() || isWriteRoutes();
-  };
+  }
 
   QString getName() const
   {
     return name_;
-  };
+  }
+
   QString getDescription() const
   {
     return description_;
-  };
+  }
+
   QString getHtml() const
   {
     return html_;
-  };
+  }
+
   QStringList getExtensions() const
   {
     return extensions_;
-  };
+  }
+
   const QList<FormatOption>& getInputOptions()  const
   {
     return inputOptions_;
-  };
+  }
+
   const QList<FormatOption>& getOutputOptions() const
   {
     return outputOptions_;
-  };
+  }
 
   QList<FormatOption>* getInputOptionsRef()
   {
     return &inputOptions_;
-  };
+  }
+
   QList<FormatOption>* getOutputOptionsRef()
   {
     return &outputOptions_;
-  };
+  }
 
   bool isDeviceFormat() const
   {
     return deviceFormat_;
-  };
+  }
+
   bool isFileFormat() const
   {
     return   fileFormat_;
-  };
+  }
 
   bool isHidden() const
   {
     return hidden_;
-  };
+  }
+
   void setHidden(bool state)
   {
     hidden_ = state;
-  };
+  }
 
   void saveSettings(QSettings& settings);
   void restoreSettings(QSettings& settings);
index 6aa5cc6f8189428483f0f01038baf13c028bf3e9..6df76043d0c0a05c4fa22d7d593d8a9f3425daba 100644 (file)
@@ -33,8 +33,8 @@
 class FormatLoad
 {
 public:
-  FormatLoad() : currentLine_(0) {};
-  ~FormatLoad() {};
+  FormatLoad() : currentLine_(0) {}
+  ~FormatLoad() {}
 
   bool getFormats(QList<Format>& formatList);
 private:
index 2a876f9f1864dc9d6421b7f185ecbdbd454a1848..e5355dcc84337cdd0936dbfbf4232c02b8b695f1 100644 (file)
@@ -80,8 +80,8 @@ public:
   {
     if (localName == "wpt") {
       currentWpt = GpxWaypoint();
-      double lat = atts.value("lat").toDouble();;
-      double lon = atts.value("lon").toDouble();;
+      double lat = atts.value("lat").toDouble();
+      double lon = atts.value("lon").toDouble();
       currentWpt.setLocation(LatLng(lat, lon));
       stateStack << state;
       state = e_wpt;
@@ -101,8 +101,8 @@ public:
 
     else if (localName == "trkpt") {
       currentTrkPt = GpxTrackPoint();
-      double lat = atts.value("lat").toDouble();;
-      double lon = atts.value("lon").toDouble();;
+      double lat = atts.value("lat").toDouble();
+      double lon = atts.value("lon").toDouble();
       currentTrkPt.setLocation(LatLng(lat, lon));
       stateStack << state;
       state = e_trkpt;
@@ -116,8 +116,8 @@ public:
 
     else if (localName == "rtept") {
       currentRtePt = GpxRoutePoint();
-      double lat = atts.value("lat").toDouble();;
-      double lon = atts.value("lon").toDouble();;
+      double lat = atts.value("lat").toDouble();
+      double lon = atts.value("lon").toDouble();
       currentRtePt.setLocation(LatLng(lat, lon));
       stateStack << state;
       state = e_rtept;
@@ -132,7 +132,7 @@ public:
       //localName.toStdString().c_str(), qName.toStdString().c_str());
     }
     return true;
-  };
+  }
 
   bool endElement(const QString& /*namespaceURI*/,
                   const QString& localName,
@@ -205,13 +205,13 @@ public:
       //localName.toStdString().c_str(), qName.toStdString().c_str());
     }
     return true;
-  };
+  }
 
   bool characters(const QString& x) override
   {
     textChars = x;
     return true;
-  };
+  }
 };
 
 
index d2f1eab27afafa7a784b0f4bf1e6408bfcaeafa9..2bd1407652d4c4a3558e9fa815af06c7d6d110aa 100644 (file)
--- a/gui/gpx.h
+++ b/gui/gpx.h
 class GpxItem
 {
 public:
-  GpxItem(): visible(true) {};
-  GpxItem(bool visible): visible(visible) {};
+  GpxItem(): visible(true) {}
+  GpxItem(bool visible): visible(visible) {}
 
   void setVisible(bool b)
   {
     visible = b;
-  };
+  }
+
   bool getVisible() const
   {
     return visible;
-  };
+  }
 
 protected:
   bool visible;
@@ -53,24 +54,27 @@ class GpxRoutePoint: public GpxItem
 public:
   GpxRoutePoint(): GpxItem(), location(LatLng()), name(QString())
   {
-  };
+  }
+
   void setLocation(const LatLng& pt)
   {
     location = pt;
-  };
+  }
+
   LatLng getLocation() const
   {
     return location;
-  };
+  }
 
   void setName(const QString& s)
   {
     name = s;
-  };
+  }
+
   QString getName() const
   {
     return name;
-  };
+  }
 
 private:
   LatLng location;
@@ -81,7 +85,7 @@ private:
 class GpxRoute: public GpxItem
 {
 public:
-  GpxRoute(): GpxItem(),name(QString()), cachedLength(-1) {};
+  GpxRoute(): GpxItem(),name(QString()), cachedLength(-1) {}
 
   GpxRoute(const GpxRoute& c)
     :GpxItem(c.visible),
@@ -130,16 +134,18 @@ public:
   void setName(const QString& s)
   {
     name = s;
-  };
+  }
+
   QString getName() const
   {
     return name;
-  };
+  }
 
   void clear()
   {
     routePoints.clear();
-  };
+  }
+
   void addPoint(const GpxRoutePoint& pt)
   {
     routePoints << pt;
@@ -147,7 +153,7 @@ public:
   const QList<GpxRoutePoint>& getRoutePoints() const
   {
     return routePoints;
-  };
+  }
 
 private:
   QString name;
@@ -161,33 +167,37 @@ class GpxTrackPoint: public GpxItem
 public:
   GpxTrackPoint(): GpxItem(), location(LatLng()), elevation(0), dateTime(QDateTime())
   {
-  };
+  }
+
   void setLocation(const LatLng& pt)
   {
     location = pt;
-  };
+  }
+
   LatLng getLocation() const
   {
     return location;
-  };
+  }
 
   void setElevation(double e)
   {
     elevation = e;
-  };
+  }
+
   double getElevation() const
   {
     return elevation;
-  };
+  }
 
   void setDateTime(const QDateTime& dt)
   {
     dateTime = dt;
-  };
+  }
+
   QDateTime getDateTime() const
   {
     return dateTime;
-  };
+  }
 
 private:
   LatLng location;
@@ -199,7 +209,8 @@ private:
 class GpxTrackSegment: public GpxItem
 {
 public:
-  GpxTrackSegment() {};
+  GpxTrackSegment() {}
+
   GpxTrackSegment(const GpxTrackSegment& c): GpxItem(c.visible)
   {
     trackPoints.clear();
@@ -223,11 +234,12 @@ public:
   void clear()
   {
     trackPoints.clear();
-  };
+  }
+
   const QList<GpxTrackPoint>& getTrackPoints() const
   {
     return trackPoints;
-  };
+  }
 
 private:
   QList <GpxTrackPoint> trackPoints;
@@ -236,7 +248,7 @@ private:
 class GpxTrack: public GpxItem
 {
 public:
-  GpxTrack(): GpxItem(), number(1), name(QString()), comment(QString()), description(QString()), cachedLength(-1.0) {};
+  GpxTrack(): GpxItem(), number(1), name(QString()), comment(QString()), description(QString()), cachedLength(-1.0) {}
 
   GpxTrack(const GpxTrack& c)
     :GpxItem(c.visible),
@@ -269,43 +281,48 @@ public:
   void setNumber(int n)
   {
     number = n;
-  };
+  }
+
   int getNumber() const
   {
     return number;
-  };
+  }
 
   void setName(const QString& s)
   {
     name = s;
-  };
+  }
+
   QString getName() const
   {
     return name;
-  };
+  }
 
   void setComment(const QString& s)
   {
     comment = s;
-  };
+  }
+
   QString getComment() const
   {
     return comment;
-  };
+  }
 
   void setDescription(const QString& s)
   {
     description = s;
-  };
+  }
+
   QString getDescription() const
   {
     return description;
-  };
+  }
 
   void clear()
   {
     trackSegments.clear();
-  };
+  }
+
   void addSegment(const GpxTrackSegment& seg)
   {
     trackSegments << seg;
@@ -313,7 +330,7 @@ public:
   const QList<GpxTrackSegment>& getTrackSegments() const
   {
     return trackSegments;
-  };
+  }
 
   double length() const
   {
@@ -360,61 +377,67 @@ public:
     comment_(QString()),
     description_(QString()),
     symbol_(QString())
-  {};
+  {}
 
   void setLocation(const LatLng& pt)
   {
     location_ = pt;
-  };
+  }
+
   LatLng getLocation() const
   {
     return location_;
-  };
+  }
 
   void setElevation(double e)
   {
     elevation_ = e;
-  };
+  }
+
   double getElevation() const
   {
     return elevation_;
-  };
+  }
 
   void setName(const QString& s)
   {
     name_ = s;
-  };
+  }
+
   QString getName() const
   {
     return name_;
-  };
+  }
 
   void setComment(const QString& s)
   {
     comment_ = s;
-  };
+  }
+
   QString getComment() const
   {
     return comment_;
-  };
+  }
 
   void setDescription(const QString& s)
   {
     description_ = s;
-  };
+  }
+
   QString getDescription() const
   {
     return description_;
-  };
+  }
 
   void setSymbol(const QString& s)
   {
     symbol_ = s;
-  };
+  }
+
   QString getSymbol() const
   {
     return symbol_;
-  };
+  }
 
 private:
   LatLng location_;
@@ -429,34 +452,38 @@ private:
 class Gpx
 {
 public:
-  Gpx() {};
+  Gpx() {}
   bool read(const QString& fileName);
 
   QList <GpxWaypoint>& getWaypoints()
   {
     return wayPoints;
-  }; // nonconst
+  } // nonconst
+
   QList <GpxTrack>&    getTracks()
   {
     return tracks;
-  };
+  }
+
   QList <GpxRoute>&    getRoutes()
   {
     return routes;
-  };
+  }
 
   const QList <GpxWaypoint>& getWaypoints() const
   {
     return wayPoints;
-  };
+  }
+
   const QList <GpxTrack>& getTracks()       const
   {
     return tracks;
-  };
+  }
+
   const QList <GpxRoute>& getRoutes()       const
   {
     return routes;
-  };
+  }
 
 private:
   QList <GpxWaypoint> wayPoints;
index 1f85152d73c296376ee65b61cdf7ab023abe92cd..df5d5e50d4e4855e44bca63b8586c32bccb4dd7b 100644 (file)
@@ -1347,7 +1347,7 @@ QString MainWindow::getFormatNameForExtension(const QString& ext)
     QStringList extensions = formatList_[i].getExtensions();
     for (int j = 0; j < extensions.size(); ++j) {
       if (extensions[j] == ext) {
-        return formatList_[i].getName();;
+        return formatList_[i].getName();
       }
     }
   }
index 92fcf8761bf35ab45214dcd6d57a50df9f5a9526..cced27bfcc1f2e8e5c0ab012ab38abb851b5e17d 100644 (file)
--- a/gui/map.h
+++ b/gui/map.h
@@ -43,7 +43,7 @@ class MarkerClicker: public QObject
   Q_OBJECT
 
 public:
-  MarkerClicker(QObject* parent): QObject(parent) {};
+  MarkerClicker(QObject* parent): QObject(parent) {}
 
 public slots:
   void clickedX(int t, int i)
index 4671d2ff2b00bc205979a46230cd1ac11909d59c..8ae5806f4479337a0ddb9662dc5d1efcc5763b87 100644 (file)
@@ -123,25 +123,26 @@ ProcessWaitDialog::~ProcessWaitDialog()
 bool ProcessWaitDialog::getExitedNormally()
 {
   return (errorString_.length() == 0);
-};
+}
 
 //------------------------------------------------------------------------
 QString ProcessWaitDialog::getErrorString()
 {
   return errorString_;
-};
+}
 
 //------------------------------------------------------------------------
 int ProcessWaitDialog::getExitCode()
 {
   return ecode_;
-};
+}
 
 //------------------------------------------------------------------------
 void ProcessWaitDialog::stopClickedX()
 {
   process_->terminate();
-};
+}
+
 //------------------------------------------------------------------------
 void ProcessWaitDialog::timeoutX()
 {
@@ -157,7 +158,7 @@ void ProcessWaitDialog::timeoutX()
     timer_->stop();
     accept();
   }
-};
+}
 
 //------------------------------------------------------------------------
 void ProcessWaitDialog::errorX(QProcess::ProcessError err)
@@ -165,7 +166,7 @@ void ProcessWaitDialog::errorX(QProcess::ProcessError err)
   errorString_ = processErrorString(err);
   timer_->stop();
   accept();
-};
+}
 
 //------------------------------------------------------------------------
 void ProcessWaitDialog::finishedX(int exitCode, QProcess::ExitStatus es)
@@ -176,7 +177,7 @@ void ProcessWaitDialog::finishedX(int exitCode, QProcess::ExitStatus es)
   }
   timer_->stop();
   accept();
-};
+}
 
 
 //------------------------------------------------------------------------
@@ -205,16 +206,16 @@ void ProcessWaitDialog::readyReadStandardErrorX()
 {
   QByteArray d = process_->readAllStandardError();
   appendToText(d.data());
-};
+}
 
 //------------------------------------------------------------------------
 void ProcessWaitDialog::readyReadStandardOutputX()
 {
   QByteArray d = process_->readAllStandardOutput();
   appendToText(d.data());
-};
+}
 
 void ProcessWaitDialog::closeEvent(QCloseEvent* event)
 {
   event->ignore();
-};
+}
index 2faf6f7bf9b0de639ce3c322356e935f9b6bb5da..1cce33727852d71671b818d4b0a2288cec94932d 100644 (file)
@@ -52,7 +52,7 @@ public:
   QString getOutputString() const
   {
     return outputString_;
-  };
+  }
 
 protected:
   void closeEvent(QCloseEvent* event);
index 89770cfa3f8e6fef0d8b804d57b63368fb78d716..0f111c991978df161a5063c799ea4d3def35356d 100644 (file)
@@ -31,8 +31,8 @@
 class VarSetting
 {
 public:
-  VarSetting() {};
-  virtual ~VarSetting() {};
+  VarSetting() {}
+  virtual ~VarSetting() {}
 
   virtual void saveSetting(QSettings&) = 0;
   virtual void restoreSetting(QSettings&) = 0;
index df113da65adb4eee1adba082277db2512e1d034b..e8a63675933664afc00e8ee4e88a5ddaa08c77c5 100644 (file)
@@ -2490,7 +2490,7 @@ static void GPS_D109_Send(UC* data, GPS_PWay way, int32* len, int protoid)
       }
     }
 
-    GPS_Util_Put_Short(p, (US) way->category);; /* D110 category */
+    GPS_Util_Put_Short(p, (US) way->category); /* D110 category */
     p += 2;
   }
 
index 15ca36914182adbeae1ca54297b31e836a00eb5c..36de12475c0a8d328fcf84256d14955bde1d407e 100644 (file)
@@ -279,7 +279,7 @@ int32 GPS_Serial_Set_Baud_Rate(gpsdevh* fd, int br)
   tio.DCBlength = sizeof(DCB);
 
   GetCommState(wsd->comport, &tio);
-  tio.BaudRate = speed;;
+  tio.BaudRate = speed;
   if (!SetCommState(wsd->comport, &tio)) {
     GPS_Serial_Error("SetCommState on port for alternate bit rate failed");
     CloseHandle(wsd->comport);
index e8f17a5df9d54dbe42249139d99aef2ae9d67488..f85e555fd1a327a4e26422de3b5d7973b6fc08eb 100644 (file)
@@ -125,7 +125,7 @@ top:
     if ((rv == 0)  && (receive_state == rs_frombulk)) {
       m1= "RET2INTR";
       m2=nullptr;
-    };
+    }
     GPS_Diag("(%-8s%s)\n", m1, m2 ? m2 : "");
   }
 
diff --git a/pcx.cc b/pcx.cc
index 11dc8e171b53d54cf39c8a89deb4c5e4a2ff8345..e3e4aa77792998191a10333dbb16e04d2d555d19 100644 (file)
--- a/pcx.cc
+++ b/pcx.cc
@@ -286,7 +286,6 @@ static void data_read() {
       } break;
       default:
         break;
-        ;
     }
   }
 }
index e1885ce0013d89845777c593294c7cc695b972fc..78d2bc7638abf3a6299519a94c596ccf5896e362 100644 (file)
--- a/route.cc
+++ b/route.cc
@@ -385,7 +385,7 @@ route_head::route_head() :
   line_width(-1),
   session(curr_session())
 {
-};
+}
 
 route_head::~route_head()
 {
index d27221a7bb5a807531728b7dc7fe233d76b5b2d0..60c5d2c8e9a3d161a7dfcd882882f2faf263781e 100644 (file)
@@ -60,6 +60,6 @@ public:
 
 };
 
-}; // namespace gpsbabel
+} // namespace gpsbabel
 
 #endif // SRC_CORE_FILE_INCLUDED_H_
index afbc7b615fad40757fdf4bd1cad17a8e48e1f23e..a1528cac5fc48e0cb1f22fa818ab2575b4f4e52f 100644 (file)
@@ -65,4 +65,4 @@ void TextStream::close()
   codec_ = nullptr;
 }
 
-}; // namespace
+} // namespace
index bcfaa64ab1c62cb0458227130fdce51523fd2b8e..d1d40b193935d3bf472b49d0b67e1efb330c65c3 100644 (file)
@@ -40,4 +40,4 @@ private:
   QTextCodec* codec_{nullptr};
 };
 
-}; // namespace
+} // namespace
index febcb8afcd7bbb58f82622ab12cc4dc837781aa3..9bd9ae91e4cb251527c363caaf88d6a5a9125def 100644 (file)
@@ -160,7 +160,7 @@ xml_run_parser(QXmlStreamReader& reader)
 
     default:
       break;
-    };
+    }
 
 readnext:
     // readNextStartElement will cause a "Premature end of document." error